Skip to content

fix(report): clarify scan output and package versions - #1614

Open
John-David Dalton (jdalton) wants to merge 1 commit into
v1.xfrom
jdalton/report-version-v1x
Open

John-David Dalton (jdalton) wants to merge 1 commit into
v1.xfrom
jdalton/report-version-v1x

Conversation

@jdalton

@jdalton John-David Dalton (jdalton) commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Make scan reports easier to read with a health summary and an aligned alert table. Label package versions as Version in Markdown and VERSION in terminal output.

Include tests that check versions, manifests, and URLs across default, version, file, and package folds in both formats. These tests incorporate the work from #1710.

Note

Minor change: Markdown reports now label this field Version instead of Introduced by. JSON output is unchanged.

@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 3 times, most recently from 0c7ab2d to 73b00b7 Compare November 11, 2026 16:41
@jdalton
John-David Dalton (jdalton) changed the base branch from fix/scan-report-text-v1x to v1.x November 11, 2026 16:41
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 2 times, most recently from c8e0f56 to c1d03f3 Compare November 11, 2026 16:55
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 2 times, most recently from 24ddac7 to a164eb5 Compare November 11, 2026 17:09
@jdalton

This comment was marked as outdated.

@cursor

cursor Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes default CLI report output and renames a Markdown table column, which may break consumers parsing Introduced by; behavior of JSON output is unchanged.

Overview
Replaces default socket scan report terminal output with a formatted plain-text report (health summary, settings, aligned alert table with URLs on separate lines) instead of dumping nested Map structures via logger.dir.

Markdown reports now label the third alert column Version (was Introduced by) and URL (was url), backed by shared flattenReportAlerts / table-formatting helpers used by both Markdown and text output. JSON report shape is unchanged.

Adds broad Vitest coverage for plain text rendering (Unicode, column alignment, empty alerts, fold modes, large reports) and updates existing Markdown snapshots.

Reviewed by Cursor Bugbot for commit a164eb5. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit a164eb5. Configure here.

@jdalton John-David Dalton (jdalton) changed the title fix(report): label package versions fix(report): clarify scan output and package versions Nov 12, 2026
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 2 times, most recently from 3e3b0c7 to b9aeceb Compare November 12, 2026 15:27

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant